Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[material-ui] Fix text-wrap line height #44464

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ChristopherJamesL
Copy link

@zannager zannager added the component: Typography The React component. label Nov 19, 2024
@aarongarciah
Copy link
Member

aarongarciah commented Nov 25, 2024

Hey @ChristopherJamesL, thanks for contributing. May I ask you why this change is needed? Some visuals or examples might help. As you can see in the visual regression tests, this is a pretty sensitive change.

@aarongarciah aarongarciah added the package: material-ui Specific to @mui/material label Nov 25, 2024
@aarongarciah aarongarciah changed the title fixed text-wrap line height #38675 [material-ui] Fix text-wrap line height Nov 25, 2024
@mui-bot
Copy link

mui-bot commented Nov 25, 2024

Netlify deploy preview

https://deploy-preview-44464--material-ui.netlify.app/

Bundle size report

Details of bundle changes (Toolpad)
Details of bundle changes

Generated by 🚫 dangerJS against d73934c

@aarongarciah aarongarciah added the status: waiting for author Issue with insufficient information label Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: Typography The React component. package: material-ui Specific to @mui/material status: waiting for author Issue with insufficient information
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants